Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make bindata.go optional, remove from repository #172

Closed
wants to merge 1 commit into from

Conversation

strk
Copy link
Member

@strk strk commented Nov 14, 2016

By default will read assets from filesystems, but can embed them
by passing the "bindata" build tag (go build -tags bindata)

Fixes #30

Print a WARNING when bindata is not embedded, and
honours GITEA_WORK_DIR, when given, for finding filesystem assets.

Includes automatic test for the dynamic "bin"data library.

NOTE: an equivalent functionality is planned to be included in a
bigger change by @tboerger currently in progress and tracked as PR #74;
this PR is smaller in scope and ready.

Please keep open until either this or #74 is merged.

@bkcsoft
Copy link
Member

bkcsoft commented Nov 14, 2016

Keep this open til #74 is merged, so people can use this before the other fix is done.

@bkcsoft bkcsoft added type/enhancement An improvement of existing functionality issue/duplicate The issue has already been reported. labels Nov 14, 2016
@codecov-io
Copy link

codecov-io commented Nov 14, 2016

Current coverage is 3.25% (diff: 31.25%)

Merging #172 into master will increase coverage by 0.22%

@@            master      #172   diff @@
========================================
  Files           33        34     +1   
  Lines         8106      8170    +64   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
+ Hits           246       266    +20   
- Misses        7840      7881    +41   
- Partials        20        23     +3   

Powered by Codecov. Last update 2ccdcda...38b713c

@strk strk force-pushed the optional-bindata branch 4 times, most recently from 8213590 to 9184ca6 Compare November 21, 2016 10:07
@strk strk force-pushed the optional-bindata branch 3 times, most recently from 8333ffc to 3d14719 Compare November 24, 2016 08:02
@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 24, 2016
@strk strk force-pushed the optional-bindata branch 5 times, most recently from 25ceda1 to 5ff07a9 Compare November 27, 2016 11:00
@tboerger tboerger added this to the 1.0.0 milestone Nov 29, 2016
@strk strk force-pushed the optional-bindata branch 3 times, most recently from b324f98 to 8f6c1e8 Compare December 2, 2016 11:06
@bkcsoft bkcsoft removed this from the 1.0.0 milestone Dec 3, 2016
By default will read assets from filesystems, but can embed them
by passing the "bindata" build tag (go build -tags bindata)

Fixes gogs#77

Print a WARNING when bindata is not embedded, and
honours GITEA_WORK_DIR, when given, for finding filesystem assets.

Include automatic test for the dynamic "bin"data library
@tboerger
Copy link
Member

tboerger commented Dec 5, 2016

Obsoleted via #354

@tboerger tboerger closed this Dec 5, 2016
@tboerger tboerger removed type/enhancement An improvement of existing functionality lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 5, 2016
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/duplicate The issue has already been reported.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Embed bindata optionally
4 participants